Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve notification banner transitions #5170

Merged
merged 1 commit into from
Sep 22, 2023

Conversation

rablador
Copy link
Contributor

@rablador rablador commented Sep 20, 2023

Notification banner animations might get choppy if many notifications are fired quickly after each other.


This change is Reviewable

@rablador rablador added the iOS Issues related to iOS label Sep 20, 2023
@linear
Copy link

linear bot commented Sep 20, 2023

IOS-155 Improve notification banner transitions

Notification banner animations might get choppy if many notifications are fired quickly after each other. A solution to this could be to stop the property animator and restart a new one from the previous position.

@rablador rablador force-pushed the improve-notification-banner-transitions-ios-155 branch from 896468a to 1555cf2 Compare September 22, 2023 09:42
Copy link
Contributor Author

@rablador rablador left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was a proposal to use newly added API callback viewIsAppearing(_:) for calling view.layoutIfNeeded() instead of currently used toggleBanner(). This causes regression and introduces horisontal expansion. Leaving it where it's currently at and not setting bannerView to hidden removes all choppy or weird animation behavior that I could find.

Reviewable status: 0 of 1 files reviewed, all discussions resolved

@rablador rablador added the bug label Sep 22, 2023
@rablador rablador marked this pull request as ready for review September 22, 2023 09:49
Copy link
Contributor

@buggmagnet buggmagnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice !

Reviewed all commit messages.
Reviewable status: 0 of 1 files reviewed, all discussions resolved

@buggmagnet buggmagnet force-pushed the improve-notification-banner-transitions-ios-155 branch from 1555cf2 to b85f1b2 Compare September 22, 2023 13:17
@buggmagnet buggmagnet merged commit 5697876 into main Sep 22, 2023
4 checks passed
@buggmagnet buggmagnet deleted the improve-notification-banner-transitions-ios-155 branch September 22, 2023 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug iOS Issues related to iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants